[webgpu] Enable profiling for graph capture#27058
Conversation
There was a problem hiding this comment.
Pull request overview
This PR enables profiling support during graph capture and replay in the WebGPU execution provider. Previously, profiling was explicitly prevented during graph capture mode.
Changes:
- Removed the enforcement that blocked profiling during graph capture
- Added profiling data storage in captured commands for replay
- Wrapped graph replay with profiling start/collect calls
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| onnxruntime/core/providers/webgpu/webgpu_execution_provider.cc | Added profiling calls around graph replay to enable profiling data collection |
| onnxruntime/core/providers/webgpu/webgpu_context.h | Added optional profiling data field to CapturedCommandInfo and new PendingKernelInfo constructor for replay |
| onnxruntime/core/providers/webgpu/webgpu_context.cc | Stores profiling data during capture and restores it during replay; removed profiling restriction |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@guschmue @fs-eire @xiaofeihan1 Please take a look, thanks. |
|
please merge the latest main again to include all fixes to the pipelines. |
4 participants
Description
Motivation and Context